auber - Technical Training #1058
Conversation
|
|
barracudapps
left a comment
barracudapps
left a comment
There was a problem hiding this comment.
Please review your commit message and its title to follow Odoo's Git Guidelines
FYI
[REV] stands for Revert. Here, you didn't revert anything, you didn't "erase" something but changes/added. IMHO [FIX] would've been a better tag
aurb9
force-pushed
the
19.0-tutorials-auber
branch
2 times, most recently
from
69edf2e to
f578c54
Compare
barracudapps
left a comment
barracudapps
left a comment
There was a problem hiding this comment.
Hi @aurb9
Can you please consider my nit and apply it wherever it suits ?
Thanks!
barracudapps
left a comment
barracudapps
left a comment
There was a problem hiding this comment.
Quick review requiring some changes (nit)
|
@aurb9 |
Solved -- was importing the views in the wrong order |
barracudapps
left a comment
barracudapps
left a comment
There was a problem hiding this comment.
LGTM 👍
Please adapt your PR title and message to summarize your work.
Then, you can squash your commits and mark the PR as ready 🚀
This is for a common R&D flow having pre-review requirements, nothing to do for the training 😉 |
This provides an implementation for a module allowing people to manage their properties, people to place offers and alters the users' view to include the list of properties being sold by the person
aurb9
force-pushed
the
19.0-tutorials-auber
branch
from
ce00226 to
a7e4365
Compare
3 participants
This provides an implementation for a module allowing people to manage their properties, people to place offers and alters the users' view to include the list of properties being sold by the person